ui package upgrades by DaanHoogland · Pull Request #13632 · apache/cloudstack

DaanHoogland · 2026-07-16T14:10:54Z

Description

This PR...

Fixes: #13631

Types of changes

Breaking change (fix or feature that would cause existing functionality to change)
New feature (non-breaking change which adds functionality)
Bug fix (non-breaking change which fixes an issue)
Enhancement (improves an existing feature and functionality)
Cleanup (Code refactoring and cleanup, that may add test cases)
Build/CI
Test (unit or integration test code)

Feature/Enhancement Scale or Bug Severity

Feature/Enhancement Scale

Major
Minor

Bug Severity

Screenshots (if appropriate):

How Has This Been Tested?

How did you try to break this feature and the system with this change?

Copilot

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Copilot

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

codecov · 2026-07-16T14:23:22Z

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 20.81%. Comparing base (f8f3848) to head (913dec4).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff              @@
##               main   #13632      +/-   ##
============================================
+ Coverage     20.78%   20.81%   +0.02%     
- Complexity    21940    21943       +3     
============================================
  Files          6496     6496              
  Lines        591822   591474     -348     
  Branches      72615    72617       +2     
============================================
+ Hits         123015   123090      +75     
+ Misses       455452   455006     -446     
- Partials      13355    13378      +23

Flag	Coverage Δ
uitests	`3.78% <ø> (-0.10%)`	⬇️
unittests	`22.11% <ø> (+0.02%)`	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

github-actions · 2026-07-16T15:25:01Z

🔴 Test Coverage Grade: `D` — Marginal

Metric	Value
Line coverage	24.50%
Branch coverage	18.66%

Grade Scale

Grade	Line Coverage	Meaning
🟢 A	≥ 80%	Excellent - this code sleeps well at night 😴
🟡 B	60-79%	Good - almost there, don't stop now 😉
🟠 C	40-59%	Acceptable - your code is wearing a seatbelt, but no airbags 😬
🔴 D	20-39%	Marginal - boldly shipping where no test has gone before 🖖
⛔ F	< 20%	Failing - tests? what tests? 🔥

Branch coverage is shown as a secondary signal. Grade is determined by line coverage.
View full Actions run

Copilot

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

DaanHoogland · 2026-07-16T16:32:55Z

@blueorangutan ui

blueorangutan · 2026-07-16T16:34:04Z

@DaanHoogland a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

blueorangutan · 2026-07-16T16:56:03Z

UI build: ✔️
Live QA URL: https://qa.cloudstack.cloud/simulator/pr/13632 (QA-JID-961)

github-actions · 2026-07-16T17:22:46Z

🔴 Test Coverage Grade: `D` — Marginal

Metric	Value
Line coverage	24.50%
Branch coverage	18.66%

Grade Scale

Grade	Line Coverage	Meaning
🟢 A	≥ 80%	Excellent - this code sleeps well at night 😴
🟡 B	60-79%	Good - almost there, don't stop now 😉
🟠 C	40-59%	Acceptable - your code is wearing a seatbelt, but no airbags 😬
🔴 D	20-39%	Marginal - boldly shipping where no test has gone before 🖖
⛔ F	< 20%	Failing - tests? what tests? 🔥

Branch coverage is shown as a secondary signal. Grade is determined by line coverage.
View full Actions run

Copilot

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

github-actions · 2026-07-17T14:31:52Z

🔴 Test Coverage Grade: `D` — Marginal

Metric	Value
Line coverage	24.50%
Branch coverage	18.66%

Grade Scale

Grade	Line Coverage	Meaning
🟢 A	≥ 80%	Excellent - this code sleeps well at night 😴
🟡 B	60-79%	Good - almost there, don't stop now 😉
🟠 C	40-59%	Acceptable - your code is wearing a seatbelt, but no airbags 😬
🔴 D	20-39%	Marginal - boldly shipping where no test has gone before 🖖
⛔ F	< 20%	Failing - tests? what tests? 🔥

Branch coverage is shown as a secondary signal. Grade is determined by line coverage.
View full Actions run

DaanHoogland · 2026-07-20T10:05:31Z

@blueorangutan ui

blueorangutan · 2026-07-20T10:06:04Z

@DaanHoogland a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

blueorangutan · 2026-07-20T10:28:15Z

UI build failed: ✖️
(SL-JID-965)

github-actions · 2026-07-20T17:07:48Z

🔴 Test Coverage Grade: `D` — Marginal

Metric	Value
Line coverage	24.50%
Branch coverage	18.66%

Grade Scale

Grade	Line Coverage	Meaning
🟢 A	≥ 80%	Excellent - this code sleeps well at night 😴
🟡 B	60-79%	Good - almost there, don't stop now 😉
🟠 C	40-59%	Acceptable - your code is wearing a seatbelt, but no airbags 😬
🔴 D	20-39%	Marginal - boldly shipping where no test has gone before 🖖
⛔ F	< 20%	Failing - tests? what tests? 🔥

Branch coverage is shown as a secondary signal. Grade is determined by line coverage.
View full Actions run

DaanHoogland · 2026-07-20T18:34:01Z

@blueorangutan ui

blueorangutan · 2026-07-20T18:35:04Z

@DaanHoogland a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

blueorangutan · 2026-07-20T18:56:25Z

UI build: ✔️
Live QA URL: https://qa.cloudstack.cloud/simulator/pr/13632 (QA-JID-967)

github-actions · 2026-07-20T19:33:45Z

🔴 Test Coverage Grade: `D` — Marginal

Metric	Value
Line coverage	24.50%
Branch coverage	18.66%

Grade Scale

Grade	Line Coverage	Meaning
🟢 A	≥ 80%	Excellent - this code sleeps well at night 😴
🟡 B	60-79%	Good - almost there, don't stop now 😉
🟠 C	40-59%	Acceptable - your code is wearing a seatbelt, but no airbags 😬
🔴 D	20-39%	Marginal - boldly shipping where no test has gone before 🖖
⛔ F	< 20%	Failing - tests? what tests? 🔥

Branch coverage is shown as a secondary signal. Grade is determined by line coverage.
View full Actions run

Copilot

Pull request overview

Copilot reviewed 9 out of 10 changed files in this pull request and generated 2 comments.

+NODE_MAJOR=$(node -p "process.versions.node.split('.')[0]")
+if [ "$NODE_MAJOR" -ge 17 ]; then
+  export NODE_OPTIONS="--openssl-legacy-provider ${NODE_OPTIONS}"
+fi


github-actions · 2026-07-21T09:31:00Z

🔴 Test Coverage Grade: `D` — Marginal

Metric	Value
Line coverage	24.50%
Branch coverage	18.66%

Grade Scale

Grade	Line Coverage	Meaning
🟢 A	≥ 80%	Excellent - this code sleeps well at night 😴
🟡 B	60-79%	Good - almost there, don't stop now 😉
🟠 C	40-59%	Acceptable - your code is wearing a seatbelt, but no airbags 😬
🔴 D	20-39%	Marginal - boldly shipping where no test has gone before 🖖
⛔ F	< 20%	Failing - tests? what tests? 🔥

Branch coverage is shown as a secondary signal. Grade is determined by line coverage.
View full Actions run

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

webpack 4 uses the MD4 hash algorithm which OpenSSL 3 (Node 17+) no longer provides by default, causing ERR_OSSL_EVP_UNSUPPORTED. NODE_OPTIONS is used (rather than a direct node flag) so the setting is inherited by all child processes that webpack spawns during the build. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Three independent issues surfaced after the recent package upgrades and were causing the UI Build workflow to fail: - The Node 17+/OpenSSL 3 fix (NODE_OPTIONS='--openssl-legacy-provider') crashes on Node 16, which is what CI actually uses, since that flag doesn't exist without OpenSSL 3. Move it into a wrapper script that only sets it when the running Node actually needs it. - @vue/compiler-sfc 3.5.40 (up from 3.2.37) now only enables the jsx babel parser plugin for <script lang="jsx">/<script setup lang="tsx"> blocks; it silently discards a 'jsx' entry from user-supplied babelParserPlugins otherwise. Views using inline JSX in a plain <script> block (already relying on cli-service's babelParserPlugins config) now fail to compile, surfacing as an unrelated-looking "Cannot read properties of null (reading 'content')" since vue-loader swallows the real compileScript error. Tag those <script> blocks lang="jsx" explicitly. - vue-router 4.6.4 and minio 7.1.3 (with its fast-xml-parser dependency) ship modern syntax (class fields, optional chaining) in both their ESM and CJS builds, which webpack 4's parser can't handle. Add them to transpileDependencies so babel processes them like first-party source. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

npm run serve/start/build now set --openssl-legacy-provider automatically via node-options-wrapper.sh, only when the running Node actually needs it. The old manual instruction is not just redundant but actively harmful: a contributor who exports it persistently and later runs on Node 16 (what CI pins) crashes node outright, since that flag doesn't exist without OpenSSL 3. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

DaanHoogland · 2026-11-11T07:17:08Z

@blueorangutan ui

Copilot

🟢 Approval recommended

No active review issues remain, and the dependency and build-tooling updates are documented as addressing Node.js 24 compatibility across the UI stack.

1 open issue

In export NODE_OPTIONS="--openssl-legacy-provider ${NODE_OPTIONS}", when NODE_OPTIONS is empty…

1 resolved since last review

Sass upgrade requires a newer Node version in UI CI

_{Review effort: Lite}

sonarqubecloud · 2026-11-11T08:39:25Z

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Copilot AI lite review requested due to automatic review settings July 16, 2026 14:10

boring-cyborg Bot added the component:UI label Jul 16, 2026

Copilot AI reviewed Jul 16, 2026

Copilot AI review requested due to automatic review settings July 16, 2026 14:13

DaanHoogland force-pushed the ghi13631-ui-package-upgrades branch from f22a38a to 2356504 Compare July 16, 2026 14:13

Copilot AI reviewed Jul 16, 2026

DaanHoogland added this to the 4.24.0 milestone Jul 16, 2026

Copilot AI review requested due to automatic review settings July 16, 2026 16:15

Copilot AI reviewed Jul 16, 2026

Copilot AI review requested due to automatic review settings July 17, 2026 13:20

Copilot AI reviewed Jul 17, 2026

Copilot AI review requested due to automatic review settings July 21, 2026 08:24

Copilot started reviewing on behalf of DaanHoogland July 21, 2026 08:32

View session

Copilot AI reviewed Jul 21, 2026

View reviewed changes

Comment thread ui/node-options-wrapper.sh

Comment on lines +23 to +26

NODE_MAJOR=$(node -p "process.versions.node.split('.')[0]")

if [ "$NODE_MAJOR" -ge 17 ]; then

export NODE_OPTIONS="--openssl-legacy-provider ${NODE_OPTIONS}"

fi

Comment thread ui/package.json Resolved

DaanHoogland and others added 22 commits November 10, 2026 16:59

ui: upgrade vue + @vue/compiler-sfc 3.2.37 -> 3.5.40

0a5acce

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

ui: upgrade @vue/test-utils 2.0.0 -> 2.4.11

938f1cc

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

ui: upgrade babel-plugin-import 1.13.5 -> 1.13.8

5fd4e94

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

ui: upgrade chartjs-adapter-moment 1.0.0 -> 1.0.1

4bdd805

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

ui: upgrade core-js 3.23.3 -> 3.49.0

dfc51e0

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

ui: upgrade cronstrue 2.26.0 -> 2.59.0

1547d5c

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

ui: upgrade eslint-plugin-import 2.26.0 -> 2.32.0

28a1c00

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

ui: upgrade minio 7.0.33 -> 7.1.3

7c46c82

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

ui: upgrade moment-timezone 0.5.43 -> 0.5.48

fd0ede9

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

ui: upgrade sass 1.53.0 -> 1.101.0

f0b7a0d

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

ui: upgrade vue-cropper 1.0.3 -> 1.1.4

943346c

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

ui: upgrade vue-qrious 3.1.0 -> 3.1.1

6da83eb

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

ui: upgrade vue-router 4.0.16 -> 4.6.4

23dc284

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

ui: upgrade vue3-google-login 2.0.26 -> 2.1.4

6bec042

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

ui: upgrade vuex 4.0.2 -> 4.1.0

605657d

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

ui: upgrade webpack 4.46.0 -> 4.47.0

9329537

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Address comments

a084a5c

ui: build image on node:24-bookworm

735d367

ui: build workflow on v24

913dec4

DaanHoogland force-pushed the ghi13631-ui-package-upgrades branch from a66d7f4 to 913dec4 Compare November 11, 2026 07:13

Copilot AI lite review requested due to automatic review settings November 11, 2026 07:13

boring-cyborg Bot added the component:integration label Nov 11, 2026

Copilot started reviewing on behalf of DaanHoogland November 11, 2026 07:14

View session

github-actions Bot removed the status:has-conflicts label Nov 11, 2026

Copilot AI reviewed Nov 11, 2026

View reviewed changes

Conversation

DaanHoogland commented Jul 16, 2026

Description

Types of changes

Feature/Enhancement Scale or Bug Severity

Feature/Enhancement Scale

Bug Severity

Screenshots (if appropriate):

How Has This Been Tested?

How did you try to break this feature and the system with this change?

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

codecov Bot commented Jul 16, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

github-actions Bot commented Jul 16, 2026

🔴 Test Coverage Grade: D — Marginal

Grade Scale

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

DaanHoogland commented Jul 16, 2026

Uh oh!

blueorangutan commented Jul 16, 2026

Uh oh!

blueorangutan commented Jul 16, 2026

Uh oh!

github-actions Bot commented Jul 16, 2026

🔴 Test Coverage Grade: D — Marginal

Grade Scale

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

github-actions Bot commented Jul 17, 2026

🔴 Test Coverage Grade: D — Marginal

Grade Scale

Uh oh!

DaanHoogland commented Jul 20, 2026

Uh oh!

blueorangutan commented Jul 20, 2026

Uh oh!

blueorangutan commented Jul 20, 2026

Uh oh!

github-actions Bot commented Jul 20, 2026

🔴 Test Coverage Grade: D — Marginal

Grade Scale

Uh oh!

DaanHoogland commented Jul 20, 2026

Uh oh!

blueorangutan commented Jul 20, 2026

Uh oh!

blueorangutan commented Jul 20, 2026

Uh oh!

github-actions Bot commented Jul 20, 2026

🔴 Test Coverage Grade: D — Marginal

Grade Scale

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

github-actions Bot commented Jul 21, 2026

🔴 Test Coverage Grade: D — Marginal

Grade Scale

Uh oh!

DaanHoogland commented Nov 11, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

🟢 Approval recommended

Uh oh!

codecov Bot commented Jul 16, 2026 •

edited

Loading

🔴 Test Coverage Grade: `D` — Marginal

🔴 Test Coverage Grade: `D` — Marginal

🔴 Test Coverage Grade: `D` — Marginal

🔴 Test Coverage Grade: `D` — Marginal

🔴 Test Coverage Grade: `D` — Marginal

🔴 Test Coverage Grade: `D` — Marginal